Removes unnecessary rollback on error. The existing call to Session.remove()
would already do this if there was an open transaction.

The Sqlalchemy docs say (sqlalchemy.scoping.scoped_session.close())

"This expunges all ORM objects associated with this
 :class:`_orm.Session`, ends any transaction in progress and
 :term:`releases` any :class:`_engine.Connection` objects which this
 :class:`_orm.Session` itself has checked out from associated
 :class:`_engine.Engine` objects. The operation then leaves the
 :class:`_orm.Session` in a state which it may be used again."

…b-rollback

Change to flask sqlalchemy db session close